Skip to content

feat(asgi): Set user_agent.original for inbound filters - #7794

Merged
sentrivana merged 4 commits into
masterfrom
ivana/user-agent-original-2
Oct 1, 2026
Merged

sentrivana merged 4 commits into
masterfrom
ivana/user-agent-original-2

Conversation

@sentrivana

@sentrivana sentrivana commented Sep 30, 2026 •

Copy link
Copy Markdown
Contributor

Part 2 of https://linear.app/getsentry/issue/PY-2898/implement-user-agentoriginal

Also deduped some ASGI app setup code in the tests

@sentrivana
sentrivana force-pushed the ivana/user-agent-original-2 branch from 457133c to 33e3b86 Compare September 30, 2026 13:01
@sentrivana
sentrivana force-pushed the ivana/user-agent-original-2 branch from 33e3b86 to 4695dc0 Compare September 30, 2026 13:02
@sentrivana sentrivana changed the title Ivana/user agent original 2 feat(asgi): Set user_agent.original for inbound filters Sep 30, 2026
@sentrivana
sentrivana marked this pull request as ready for review September 30, 2026 13:12
@sentrivana
sentrivana requested a review from a team as a code owner September 30, 2026 13:12
@github-actions

Copy link
Copy Markdown
Contributor

Codecov Results 📊

✅ 131396 passed | ⏭️ 7185 skipped | Total: 138581 | Pass Rate: 94.82% | Execution Time: 443m 51s

📊 Comparison with Base Branch

Metric Change
Total Tests 📈 +3804
Passed Tests 📈 +3807
Failed Tests 📉 -3
Skipped Tests —

➖ Removed Tests (2)

View removed tests
  • test_binary_content_encoding_mixed_content[False-False]
    • File: tests.integrations.litellm.test_litellm
  • test_cache_spans_decorator[True]
    • File: tests.integrations.django.test_cache_module

All tests are passing successfully.

✅ Patch coverage is 100.00%. Project has 2556 uncovered lines.


Generated by Codecov Action

@sentrivana
sentrivana merged commit 564c6c1 into master Oct 1, 2026
166 checks passed
@sentrivana
sentrivana deleted the ivana/user-agent-original-2 branch October 1, 2026 09:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants